Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #10 #11

Closed
wants to merge 1 commit into from
Closed

Fix #10 #11

wants to merge 1 commit into from

Conversation

alorma
Copy link

@alorma alorma commented Feb 19, 2019

Trying to implement #10

@EsteveAguilera
Copy link
Member

Hi @alorma! Do you think you'll be able to move on with this draft?

@alorma alorma closed this Jun 10, 2019
@EsteveAguilera
Copy link
Member

Oh @alorma, I was just asking!

The tests you added are a great beginning... Will you reopen this PR if I create a new branch on the project where we/you/someone else can move on with this fix?

@alorma
Copy link
Author

alorma commented Jun 11, 2019

I don't have much time to work on this now, but i will support (even coding) if not alone :$

@EsteveAguilera
Copy link
Member

Great @alorma !!!

I've just created a new branch change-definition-of-mention

You can reopen this PR pointing to that new branch so we can accept it and get that great tests!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants